chore: Add agents.md#2849
Conversation
PR Reviewer Guide 🔍(Review updated until commit f04033a)Here are some key observations to aid the review process:
|
PR Code Suggestions ✨No code suggestions found for the PR. |
|
Persistent review updated to latest commit 5b975a0 |
PR Code Suggestions ✨No code suggestions found for the PR. |
…eferences, generalize SOLID principle Agent-Logs-Url: https://github.com/Infomaniak/android-kMail/sessions/79d61e2a-7ee3-4bed-9032-11ade3a9d344 Co-authored-by: FabianDevel <45429131+FabianDevel@users.noreply.github.com>
FabianDevel
force-pushed
the
add-agents.md
branch
from
200deaa to
f04033a
Compare
There was a problem hiding this comment.
Pull request overview
Adds repository and app-level AGENTS.md documentation to guide contributors through project structure, tooling, and local development norms in the Infomaniak Mail Android codebase.
Changes:
- Added top-level
AGENTS.mddescribing composite build layout and key integration points. - Added
app/AGENTS.mddocumenting app module architecture, commands, style conventions, and testing notes.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
AGENTS.md |
Top-level documentation for composite build structure, Core/App boundaries, and common commands. |
app/AGENTS.md |
App-module specific project map, local conventions, build/test commands, and style guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Persistent review updated to latest commit f04033a |
|
Failed to generate code suggestions for PR |
… AGENTS.md files Agent-Logs-Url: https://github.com/Infomaniak/android-kMail/sessions/b428b9cd-4de7-45a5-9c92-b0f35c0da982 Co-authored-by: FabianDevel <45429131+FabianDevel@users.noreply.github.com>
Head branch was pushed to by a user without write access
|
| # Build included Core modules | ||
| ./gradlew :Core:Legacy:assemble :Core:Legacy:Confetti:assemble | ||
|
|
||
| # Run all tests | ||
| ./gradlew :app:testStandardDebugUnitTest && ./gradlew Core:test | ||
| ./gradlew :app:testStandardDebugUnitTest && ./gradlew :Core:Legacy:test :Core:Legacy:Confetti:test | ||
|
|
||
| # Lint Core (uses ktlint in Core module) | ||
| ./gradlew Core:ktlintCheck | ||
| # Lint included Core modules (uses ktlint) | ||
| ./gradlew :Core:Legacy:ktlintCheck :Core:Legacy:Confetti:ktlintCheck |
There was a problem hiding this comment.
@copilot We use composite build, you should put a reference to the :build-logic that is also needed to build core
No description provided.